Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support mobile #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

support mobile #32

wants to merge 1 commit into from

Conversation

Pranz
Copy link

@Pranz Pranz commented Sep 11, 2017

Adds support for mobile.

@ma-pe
Copy link

ma-pe commented Nov 10, 2017

Thumbs up from my side! :)

@ma-pe
Copy link

ma-pe commented Nov 10, 2017

Simple Touches doesn't work anymore. I had to comment out

e.preventDefault();

in

mousedown.split(' ').forEach(function(ev) { ... }

@Pranz
Copy link
Author

Pranz commented Nov 13, 2017

@ma-pe I think that might mess up stuff depending on the browser. I found if you want dragscroll + interactive elements within there's no silver bullet and you're gonna need a bit of custom code, so I think I'll leave my branch as i for now.

Thanks for the report!

@skeddles
Copy link

skeddles commented Feb 1, 2023

i know this is super stale, but just in case anyone else finds this:

I realized you dont actually need dragscroll on mobile, it's just how mobile works by default. all you gotta do is overflow: scroll

using this version gave me a weird bug where things scroll twice as fast as they should.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants